Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: support both Fedora 37 and 38 #22

Merged
merged 4 commits into from
Jun 12, 2023
Merged

Conversation

nejch
Copy link
Contributor

@nejch nejch commented Jun 10, 2023

Related to dev-sec/ansible-collection-hardening#671, #21.

fde891f renames latest to an explicit 38 version.
7af061a brings back Fedora 37.
Currently a complete copy as AFAIK there are no changes needed from what I could see in the hardening repo.

This shouldn't be a breaking change as old tags of https://hub.docker.com/r/rndmh3ro/docker-fedora-ansible/tags will stay on Docker Hub.

/cc @rndmh3ro @dlouzan

fedora38-ansible-latest/Dockerfile Outdated Show resolved Hide resolved
@nejch
Copy link
Contributor Author

nejch commented Jun 12, 2023

Looks like the docker build assumes the DOCKERHUB_USERNAME secret even in PRs so this doesn't work out of the box from forks - I'll define it in my fork for now and then we can follow up maybe as part of #20.

Signed-off-by: Nejc Habjan <nejc.habjan@siemens.com>
Signed-off-by: Nejc Habjan <nejc.habjan@siemens.com>
Signed-off-by: Nejc Habjan <nejc.habjan@siemens.com>
Signed-off-by: Nejc Habjan <nejc.habjan@siemens.com>
@nejch
Copy link
Contributor Author

nejch commented Jun 12, 2023

Looks like the docker build assumes the DOCKERHUB_USERNAME secret even in PRs so this doesn't work out of the box from forks - I'll define it in my fork for now and then we can follow up maybe as part of #20.

@rndmh3ro sorry, looks like even secrets defined in forks don't end up propagated to PR workflows so I added 8633f08 to make PRs from forks work out of the box. Let me know if you're happy to merge like that or I can separate that into its own PR :)

@rndmh3ro rndmh3ro merged commit 7b3ca1b into dev-sec:master Jun 12, 2023
@rndmh3ro
Copy link
Member

That's great, thanks!

@nejch nejch deleted the feat/fedora38 branch June 12, 2023 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants